Skip to content

Update IssuesService.updateComment to use PATCH instead of POST#437

Merged
robbecker-wf merged 2 commits into
SpinlockLabs:masterfrom
parlough:fix/update-comment
May 20, 2026
Merged

Update IssuesService.updateComment to use PATCH instead of POST#437
robbecker-wf merged 2 commits into
SpinlockLabs:masterfrom
parlough:fix/update-comment

Conversation

@parlough
Copy link
Copy Markdown
Contributor

Updates IssuesService.updateComment to use the request type specified by GitHub.

@parlough
Copy link
Copy Markdown
Contributor Author

parlough commented May 19, 2026

@robbecker-wf When you have a chance, would you be able to take a look and publish this fix? I appreciate the help :D

@robbecker-wf
Copy link
Copy Markdown
Contributor

I'll take a look tonight!

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request aligns IssuesService.updateComment with GitHub’s REST API by switching the request method from POST to PATCH when updating an issue comment.

Changes:

  • Switch IssuesService.updateComment from postJSON to patchJSON.
  • Bump package version to 9.25.1.
  • Add a changelog entry documenting the fix.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
pubspec.yaml Bumps the package version to 9.25.1 for the fix release.
lib/src/common/issues_service.dart Updates updateComment to use PATCH via github.patchJSON(...).
CHANGELOG.md Notes the corrected request method for IssuesService.updateComment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@robbecker-wf robbecker-wf merged commit 68ed135 into SpinlockLabs:master May 20, 2026
2 checks passed
@robbecker-wf
Copy link
Copy Markdown
Contributor

ok published https://pub.dev/packages/github/versions/9.25.1

@parlough parlough deleted the fix/update-comment branch May 20, 2026 23:06
@parlough
Copy link
Copy Markdown
Contributor Author

Thanks for the quick review and publish! The fix is working well :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants